@culka reviewed 5 files and all commit messages, and made 6 comments.
Reviewable status: all files reviewed, 6 unresolved discussions (waiting on @jannebe).

export_mbtiles_tram_links.sh line 11 at r1 (raw file):

DB_TABLE_NAME="tram_links"

MML_TRAM_IMPORT_DATE="2026-01-28"

Tätä ei varmaan pitäsi kovakoodata skriptiin

generate-tram-infralinks-from-qgis.py line 2 at r1 (raw file):

# python

Yleisesti ottaen aika kamala rakenne tässä 🙈

Funktioiden määrittelyä satunnaisissa paikoissa filun keskellä, muuttujia joiden arvoja muutellaan ties missä. Ei kuitenkaan refaktoroinnin arvoista tässä käyttötapauksessa.

generate-tram-infralinks-from-qgis.py line 28 at r1 (raw file):

import uuid

Mukaan tarkastus joka estää skriptin ajon QGIS:n ulkopuolella.

if qgis.utils.iface is None:
  raise RuntimeError('Script must be run in QGIS')

generate-tram-infralinks-from-qgis.py line 142 at r1 (raw file):

        continue

    # 1) Ellipsoidal length in metres on GRS80 using source CRS

Eikös tämä riipu kumpaa koordinaatistoa käytetään? dist tyyppi alustetaan aiemmin.

README.md line 275 at r1 (raw file):

2. Open the GeoPackage file in QGIS (drag to Layers)
1. Open Python console

Aika monta ykkössteppiä tässä peräkkäin.

set_env.sh line 94 at r1 (raw file):

docker_start() {
  if ! docker container inspect "$DOCKER_CONTAINER_NAME" >/dev/null 2>&1; then
    echo "Docker container $DOCKER_CONTAINER_NAME does not exist. Call docker_run first."

Onko kommentti tässä väärin, vai eikö docker_startilla voi käynnistää dockeria enää? Kaikki vanhat skriptit käyttävät pelkkää docker_startia.

@jannebe made 2 comments.
Reviewable status: all files reviewed, 6 unresolved discussions (waiting on @culka).

export_mbtiles_tram_links.sh line 11 at r1 (raw file):

Done.

README.md line 275 at r1 (raw file):

perus .md. Lukasepa viewerissä ;)

Mutta ehkä se on kiva että perus terminaalissakin näkyy oikein ni avasin noi tohon. 😉

@culka reviewed 5 files and all commit messages, and resolved 4 discussions.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @jannebe).